-
Notifications
You must be signed in to change notification settings - Fork 46
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ceph: improve building of dev images #512
Conversation
- Set `NVMEOF_GW` for `vstart` to work with nvmeof. - Help debug ceph.repo issues. - Add new `CEPH_CLUSTER_CEPH_REPO_BASEURL` alternative to setting `CEPH_BRANCH` and `CEPH_SHA`. - Fix setting `CEPH_CLUSTER_CEPH_REPO_BASEURL` in `Makefile` not to override value if externally set (via .env). Signed-off-by: Ernesto Puerta <[email protected]>
7c3dc0c
to
742bba9
Compare
BTW, @baum this change broke support for |
@epuertat |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm 🖖
labels: | ||
io.ceph.nvmeof: | ||
environment: | ||
TOUCHFILE: /tmp/ceph.touch | ||
NVMEOF_GW: ${NVMEOF_IP_ADDRESS}:${NVMEOF_GW_PORT} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How is the NVMEOF_GW variable utilized? And what occurs in a multi-gateway deployment?
NVMEOF_GW
forvstart
to work with nvmeof.CEPH_CLUSTER_CEPH_REPO_BASEURL
alternative to settingCEPH_BRANCH
andCEPH_SHA
.CEPH_CLUSTER_CEPH_REPO_BASEURL
inMakefile
not to override value if externally set (via .env).